-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Update documentation for TypeScript webpack usage #7550
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update documentation for TypeScript webpack usage #7550
Conversation
@alexander-akait can you please verify this and deploy it |
@snitin315 can you please check this PR and if there is no problem can you please merge it |
src/content/guides/typescript.mdx
Outdated
@@ -129,6 +165,8 @@ Note that if you're already using [`babel-loader`](https://github.com/babel/babe | |||
|
|||
To learn more about source maps, see the [development guide](/guides/development). | |||
|
|||
To learn more about source maps, see the [development guide](/guides/development). | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why it was duplicated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will fix that might be typical human error sorry for that
@Vansh5632 Looks good, let's remove duplication |
@Vansh5632 is attempting to deploy a commit to the OpenJS Foundation Team on Vercel. A member of the Team first needs to authorize it. |
removed this please check this and merge this PR |
src/content/guides/typescript.mdx
Outdated
@@ -115,6 +115,42 @@ Now lets change the import of `lodash` in our `./index.ts` due to the fact that | |||
|
|||
T> To make imports do this by default and keep `import _ from 'lodash';` syntax in TypeScript, set `"allowSyntheticDefaultImports" : true` and `"esModuleInterop" : true` in your **tsconfig.json** file. This is related to TypeScript configuration and mentioned in our guide only for your information. | |||
|
|||
## Ways to Use TypeScript in `webpack.config.js` | |||
|
|||
There are 5 ways to use TypeScript in `webpack.config.js`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh let's use .ts
here too to avoid misleading, missing this 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think there are no more add ons if any let me know so i can do them in one commit 😊
Thanks |
fixes #7537
added the required changes in the docs
image:
